Skip to content

Tighten history button spacing#1219

Draft
thomaspblock wants to merge 4 commits into
mainfrom
fix-history-button-spacing
Draft

Tighten history button spacing#1219
thomaspblock wants to merge 4 commits into
mainfrom
fix-history-button-spacing

Conversation

@thomaspblock

Copy link
Copy Markdown
Collaborator

Summary

  • Make the top-chrome history back/forward buttons narrower so they read as a tighter unit
  • Keep the sidebar toggle at its existing size
  • Add a screenshot regression covering the compact history control pair

Screenshots

Posted inline in the Buzz thread.

Test plan

  • pnpm --dir desktop typecheck
  • pnpm --dir desktop check
  • pnpm --dir desktop build
  • pnpm --dir desktop exec playwright test history-icons-screenshots.spec.ts --project=smoke

Co-authored-by: Thomas Petersen <thomasp@squareup.com>
Signed-off-by: Thomas Petersen <thomasp@squareup.com>
Co-authored-by: Thomas Petersen <thomasp@squareup.com>
Signed-off-by: Thomas Petersen <thomasp@squareup.com>
Co-authored-by: Thomas Petersen <thomasp@squareup.com>
Signed-off-by: Thomas Petersen <thomasp@squareup.com>
…acing

Co-authored-by: Thomas Petersen <thomasp@squareup.com>
Signed-off-by: Thomas Petersen <thomasp@squareup.com>

# Conflicts:
#	desktop/src/app/AppTopChrome.tsx
#	desktop/src/features/projects/ui/ProjectDetailScreen.tsx
#	desktop/src/features/projects/ui/ProjectsView.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant